Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add WETH on Optimism #554

Merged
merged 4 commits into from Sep 13, 2022
Merged

fix: add WETH on Optimism #554

merged 4 commits into from Sep 13, 2022

Conversation

pemulis
Copy link
Contributor

@pemulis pemulis commented Sep 11, 2022

Signed-off-by: John Shutt pemulis@users.noreply.github.com

This UMIP will add the WETH token on Optimism as an approved collateral currency.

https://discourse.umaproject.org/t/add-weth-as-collateral-currency-on-optimism/1803

Signed-off-by: John Shutt <pemulis@users.noreply.github.com>
Copy link
Contributor

@Reinis-FRP Reinis-FRP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe should use the opportunity and add WETH as collateral on all the other supported networks?

@@ -0,0 +1,38 @@
**UMIP #** - tbd

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this go in as a full UMIP or as a UPP to UMIP 10?
https://github.com/UMAprotocol/UMIPs/blob/master/UMIPs/umip-10.md

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's traditionally a umip if it is a new currency like this. UPPs really only for contract parameter or final fee changes (we really shouldn't have a "UPP" imo since the onchain process is exactly the same and it just introduces confusion!)

Signed-off-by: Reinis Martinsons <reinis@umaproject.org>
Signed-off-by: Reinis Martinsons <reinis@umaproject.org>
Reinis-FRP
Reinis-FRP previously approved these changes Sep 13, 2022
Signed-off-by: Reinis Martinsons <reinis@umaproject.org>
@smb2796 smb2796 merged commit a94fbaf into master Sep 13, 2022
@smb2796 smb2796 deleted the umip-weth branch September 13, 2022 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants